workspace: Make sure to sort the last row in the overview

We added special code to sort each row in the overview so that
windows were less likely to cross lines, but the awkward control
flow meant that everything but the last row got sorted.

https://bugzilla.gnome.org/show_bug.cgi?id=707197

diff --git a/js/ui/workspace.js b/js/ui/workspace.js
index a2716efd96b2951026098aba049a270860a0d532..03446ba27b9f07b2d17151593d05aa18583ce530 100644 (file)
--- a/js/ui/workspace.js
+++ b/js/ui/workspace.js
@@ -876,7 +876,6 @@ const UnalignedLayoutStrategy = new Lang.Class({
                     row.windows.push(window);
                     row.fullWidth += width;
                 } else {
-                    this._sortRow(row);
                     break;
                 }
             }
@@ -886,6 +885,8 @@ const UnalignedLayoutStrategy = new Lang.Class({
         let maxRow;
         for (let i = 0; i < numRows; i++) {
             let row = rows[i];
+            this._sortRow(row);
+
             if (!maxRow || row.fullWidth > maxRow.fullWidth)
                 maxRow = row;
             gridHeight += row.fullHeight;